Skip to content

fix(bitcoind): safer default probes and deterministic helper rpc config#8945

Merged
openoms merged 1 commit into
mainfrom
fix/bitcoind-probe-safety
May 18, 2026
Merged

fix(bitcoind): safer default probes and deterministic helper rpc config#8945
openoms merged 1 commit into
mainfrom
fix/bitcoind-probe-safety

Conversation

@openoms
Copy link
Copy Markdown
Contributor

@openoms openoms commented Apr 7, 2026

Summary

  • make bitcoind probe parameters fully configurable from values
  • set safer default probe values to reduce restart flapping during block index/verification
  • enforce explicit -conf usage for descriptor helper bitcoin-cli calls

Comparison (old -> new defaults)

  • startup failureThreshold: 90 -> 180
  • startup timeoutSeconds: 1 -> 10
  • liveness failureThreshold: 3 -> 5
  • liveness timeoutSeconds: 1 -> 5
  • readiness periodSeconds: 4 -> 10
  • readiness failureThreshold: 3 -> 6
  • readiness timeoutSeconds: 1 -> 5

Why

Observed restarts were dominated by probe timeouts during startup/index phases, not OOM. These defaults provide better startup tolerance while preserving health checks.

Scope

  • charts/bitcoind/values.yaml
  • charts/bitcoind/templates/statefulset.yaml

Comparison of effective defaults:
- startup probe: failureThreshold 90 -> 180, timeout 1s -> 10s
- liveness probe: failureThreshold 3 -> 5, timeout 1s -> 5s
- readiness probe: period 4s -> 10s, failureThreshold 3 -> 6, timeout 1s -> 5s

Template behavior updates:
- probe fields now fully value-driven (period/timeout/failure)
- descriptor helper bitcoin-cli calls now always use -conf=/data/.bitcoin/bitcoin.conf
@openoms openoms force-pushed the fix/bitcoind-probe-safety branch from 5f2fce3 to 9b9832e Compare May 18, 2026 14:23
@openoms openoms requested review from k9ert and littledino2112 May 18, 2026 14:26
@openoms openoms merged commit d3b02a6 into main May 18, 2026
3 checks passed
@openoms openoms deleted the fix/bitcoind-probe-safety branch May 18, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants